Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Order has one param without spaces #5203

Closed
wants to merge 2 commits into from
Closed

Order has one param without spaces #5203

wants to merge 2 commits into from

Conversation

carlosbuenosvinos
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets none

from http://httpd.apache.org/docs/current/mod/mod_access_compat.html#order
"Ordering is one of:

Allow,Deny
First, all Allow directives are evaluated; at least one must match, or the request is rejected. Next, all Deny directives are evaluated. If any matches, the request is rejected. Last, any requests which do not match an Allow or a Deny directive are denied by default.

Deny,Allow
First, all Deny directives are evaluated; if any match, the request is denied unless it also matches an Allow directive. Any requests which do not match any Allow or Deny directives are permitted.

Mutual-failure
This order has the same effect as Order Allow,Deny and is deprecated in its favor."

from http://httpd.apache.org/docs/current/mod/mod_access_compat.html#order
"Ordering is one of:

Allow,Deny
First, all Allow directives are evaluated; at least one must match, or the request is rejected. Next, all Deny directives are evaluated. If any matches, the request is rejected. Last, any requests which do not match an Allow or a Deny directive are denied by default.

Deny,Allow
First, all Deny directives are evaluated; if any match, the request is denied unless it also matches an Allow directive. Any requests which do not match any Allow or Deny directives are permitted.

Mutual-failure
This order has the same effect as Order Allow,Deny and is deprecated in its favor."
@cordoval
Copy link
Contributor

👍 😊

@javiereguiluz
Copy link
Member

@carlosbuenosvinos thanks for fixing this issue and for providing the reference documentation.

I have a minor question: given how strict Apache is with white spaces, should we also use Allow,Deny instead of allow,deny or is it case insensitive? Thanks.

@xabbuh
Copy link
Member

xabbuh commented Apr 22, 2015

Thank you for noticing and for providing a patch @carlosbuenosvinos. However, someone was already faster. So I'm going to close this in favor of #5197.

@xabbuh xabbuh closed this Apr 22, 2015
@carlosbuenosvinos
Copy link
Contributor Author

"Directives in the configuration files are case-insensitive, but arguments to directives are often case sensitive", however, in case of order, for me it works on every server. I'll fix it thought.

@carlosbuenosvinos
Copy link
Contributor Author

You were too fast, I fixed Allow,Deny case too. But, up to you :)

@wouterj
Copy link
Member

wouterj commented May 10, 2015

Thank you Carlos. I decided to merge this one as this had both fixes (removed space and a capitialized value). Great job!

wouterj added a commit that referenced this pull request May 10, 2015
This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #5203).

Discussion
----------

Order has one param without spaces

| Q                  | A
| ----------------- | ---
| Doc fix?        | yes
| New docs?   | no
| Applies to     | all
| Fixed tickets | none

from http://httpd.apache.org/docs/current/mod/mod_access_compat.html#order
"Ordering is one of:

Allow,Deny
First, all Allow directives are evaluated; at least one must match, or the request is rejected. Next, all Deny directives are evaluated. If any matches, the request is rejected. Last, any requests which do not match an Allow or a Deny directive are denied by default.

Deny,Allow
First, all Deny directives are evaluated; if any match, the request is denied unless it also matches an Allow directive. Any requests which do not match any Allow or Deny directives are permitted.

Mutual-failure
This order has the same effect as Order Allow,Deny and is deprecated in its favor."

Commits
-------

fe77d49 Order has one param without spaces
@wouterj wouterj closed this May 10, 2015
@carlosbuenosvinos
Copy link
Contributor Author

You welcome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants